Skip to content

sima0_17_001: Update co2diag standard name - #528

Open
jimmielin wants to merge 5 commits into
ESCOMP:developmentfrom
jimmielin:hplin/co2_diagnostic_export
Open

sima0_17_001: Update co2diag standard name#528
jimmielin wants to merge 5 commits into
ESCOMP:developmentfrom
jimmielin:hplin/co2_diagnostic_export

Conversation

@jimmielin

@jimmielin jimmielin commented Jul 22, 2026

Copy link
Copy Markdown
Collaborator

Tag name (required for release branches):
Originator(s): @jimmielin
AI tools used (if applicable; please also add the "AI-generated code" label to the PR):
What: claude-fable:5
How: wrote the companion PR

Description (include the issue title, and the keyword ['closes', 'fixes', 'resolves'] followed by the issue number):

Describe any changes made to build system:

Describe any changes made to the namelist:

List any changes to the defaults for the input datasets (e.g. boundary datasets):

List all files eliminated and why:

List all files added and what they do:

List all existing files that have been modified, and describe the changes:
(Helpful git command: git diff --name-status development...<your_branch_name>)
M src/data/registry.xml
add standard name for co2diag

If there are new failures (compared to the test/existing-test-failures.txt file),
have them OK'd by the gatekeeper, note them here, and add them to the file.
If there are baseline differences, include the test and the reason for the
diff. What is the nature of the change? Roundoff?

derecho/intel/aux_sima:

derecho/gnu/aux_sima:

derecho/nvhpc/aux_sima (test is run via Github workflow. Only run the test manually if we need to save new baselines):

  SMS_Ln2.ne3pg3_ne3pg3_mg37.FPHYStest.derecho_nvhpc.cam-outfrq_rrtmgp_derecho_gpu (Overall: NLFAIL) details:
    FAIL SMS_Ln2.ne3pg3_ne3pg3_mg37.FPHYStest.derecho_nvhpc.cam-outfrq_rrtmgp_derecho_gpu NLCOMP
  - extra NL variable: scale_dry_air_mass (appears to be intro'd in sima0_17_000 but nvhpc baseline missing for that tag.)

If this changes climate describe any run(s) done to evaluate the new
climate in enough detail that it(they) could be reproduced:

CAM-SIMA date used for the baseline comparison tests if different than latest:

…co2diag_tbd to land model.

Provenance: FIX-31 octopus
Assisted-by: claude-fable:5
@jimmielin
jimmielin requested a review from nusbaume July 22, 2026 20:26
@jimmielin jimmielin added the enhancement New feature or request label Jul 22, 2026

@nusbaume nusbaume left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @jimmielin!

@nusbaume
nusbaume requested a review from peverwhee July 30, 2026 20:15
@jimmielin jimmielin changed the title Update co2diag standard name next? Update co2diag standard name Jul 30, 2026
@jimmielin jimmielin changed the title next? Update co2diag standard name sima0_17_001: Update co2diag standard name Jul 30, 2026
@jimmielin
jimmielin requested a deployment to CI-tests-on-CIRRUS July 30, 2026 22:32 — with GitHub Actions Waiting
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants